-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(PC-29215)[API] feat: split films, series, cinema search group #13956
Conversation
0b3524f
to
a42ded5
Compare
@@ -192,17 +192,17 @@ def get_show_nodes() -> list[ShowGenre]: | |||
SEARCH_GROUP_ARTS_LOISIRS_CREATIFS = SearchGroup( | |||
technical_name="ARTS_LOISIRS_CREATIFS", | |||
label="Arts & loisirs créatifs", | |||
position=5, | |||
position=6, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
❔ question: par curiosité, comment on choisit la valeur de position ?
low logaf
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C'est les UX qui décident.
L'ancien search group était en 2, et on veut mettre les deux qui le remplacent au même endroit, donc positions 2 et 3, donc ça décale le reste de 1.
Pour vérifier qu'on fait pas de bêtise, c'est bien de tester en local et de comparer à la prod.
Je pourrai rajouter un screenshot, mais l'app ne prend pas encore en compte ces données-là (WIP).
a42ded5
to
418bfc1
Compare
bc27b9a
to
7cc1987
Compare
7cc1987
to
15a7e1c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Toujours OK pour moi 😄
But de la pull request
Ticket Jira (ou description si BSR) : https://passculture.atlassian.net/browse/PC-29215
Vérifications